Skip to content

Retain name and propertyName in declaration emit copies of binding patterns if property name is a keyword#50537

Merged
weswigham merged 3 commits into
microsoft:mainfrom
weswigham:declaration-emit-binding-keywords
Aug 30, 2022
Merged

Retain name and propertyName in declaration emit copies of binding patterns if property name is a keyword#50537
weswigham merged 3 commits into
microsoft:mainfrom
weswigham:declaration-emit-binding-keywords

Conversation

@weswigham

Copy link
Copy Markdown
Member

Fixes #50483

@typescript-bot typescript-bot added Author: Team For Milestone Bug PRs that fix a bug with a specific milestone labels Aug 30, 2022
Comment thread scripts/authors.js Outdated
@jakebailey

Copy link
Copy Markdown
Member

LGTM besides the old checked in file. Sorry about that; I considered leaving it gitignored for a while after I removed it so it'd be harder to do that, but decided against it.

@weswigham

Copy link
Copy Markdown
Member Author

cc @RyanCavanaugh to follow
@typescript-bot cherry-pick this into release-4.8

@typescript-bot

typescript-bot commented Aug 30, 2022

Copy link
Copy Markdown
Contributor

Heya @weswigham, I've started to run the task to cherry-pick this into release-4.8 on this PR at 9e0c039. You can monitor the build here.

@typescript-bot

Copy link
Copy Markdown
Contributor

Hey @weswigham, I've opened #50538 for you.

@weswigham weswigham requested a review from jakebailey August 30, 2022 15:53
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Milestone Bug PRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Isssue generating declaration files with object containing keywords as property

4 participants